Skip to content

Extra fixes adata layers None after anndata X unification#1123

Merged
LucaMarconato merged 3 commits intomainfrom
fix/anndata-layers-none2
May 11, 2026
Merged

Extra fixes adata layers None after anndata X unification#1123
LucaMarconato merged 3 commits intomainfrom
fix/anndata-layers-none2

Conversation

@LucaMarconato
Copy link
Copy Markdown
Member

@LucaMarconato LucaMarconato commented May 11, 2026

Extra fix (+ tests) in sanitize_table() after #1122

LucaMarconato and others added 3 commits May 8, 2026 13:33
…rage

Skip None when building new_keys so it is never passed to sanitize_name,
and preserve it verbatim in new_dict so the layers setter round-trips X
correctly on anndata >= 0.13 where X lives at layers[None].

Add test_sanitize_table_layers_preserves_x to verify X is intact after
sanitization; the None-in-layers assertion is guarded by a pre-check so
the test is valid on both anndata 0.12.x and >= 0.13.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@LucaMarconato LucaMarconato enabled auto-merge (squash) May 11, 2026 08:58
@codecov
Copy link
Copy Markdown

codecov Bot commented May 11, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.03%. Comparing base (824f960) to head (c486fae).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1123   +/-   ##
=======================================
  Coverage   92.03%   92.03%           
=======================================
  Files          51       51           
  Lines        7761     7761           
=======================================
  Hits         7143     7143           
  Misses        618      618           
Files with missing lines Coverage Δ
src/spatialdata/_core/_utils.py 100.00% <100.00%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@LucaMarconato LucaMarconato merged commit 9b79444 into main May 11, 2026
9 checks passed
@LucaMarconato LucaMarconato deleted the fix/anndata-layers-none2 branch May 11, 2026 09:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant